Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[squeezebox] Fix local cover art download failure when auth enabled #4654

Merged
merged 3 commits into from
Jan 25, 2019

Conversation

mhilbush
Copy link
Contributor

Signed-off-by: Mark Hilbush [email protected]

Fixes #4653

  • Add basic authentication to the download URL for LMS-served covert art images when LMS authentication is enabled.
  • Ensure that the password is not logged in the openHAB log file.
  • Add private to SqueezeBoxPlayerState fields.

Note that there are now a number of new SAT warnings, which were not there before. I presume this is because new SAT checks were added. I will fix these in a separate PR.

@mhilbush mhilbush requested a review from digitaldan as a code owner January 22, 2019 17:06
@mhilbush mhilbush changed the title Fix local cover art download failure when auth enabled [squeezebox] Fix local cover art download failure when auth enabled Jan 22, 2019
@openhab-bot
Copy link
Collaborator

This pull request has been mentioned on openHAB Community. There might be relevant details there:

https://community.openhab.org/t/squeezebox-binding-openhab-2-5-problems/64577/43

@openhab-bot
Copy link
Collaborator

This pull request has been mentioned on openHAB Community. There might be relevant details there:

https://community.openhab.org/t/squeezebox-binding-openhab-2-5-problems/64577/44

@mhilbush mhilbush force-pushed the squeezebox-coverart-fix branch from db4b41b to 3de238c Compare January 24, 2019 21:46
@martinvw martinvw merged commit 8efd842 into openhab:master Jan 25, 2019
@mhilbush mhilbush deleted the squeezebox-coverart-fix branch January 25, 2019 16:16
@wborn wborn added this to the 2.5 milestone Feb 28, 2019
Pshatsillo pushed a commit to Pshatsillo/openhab-addons that referenced this pull request Jun 19, 2019
…penhab#4654)

* Fix local cover art download failure when auth enabled
* Use asterisk for password display
* Update known issues

Signed-off-by: Mark Hilbush <[email protected]>
Signed-off-by: Pshatsillo <[email protected]>
ne0h pushed a commit to ne0h/openhab-addons that referenced this pull request Sep 15, 2019
…penhab#4654)

* Fix local cover art download failure when auth enabled
* Use asterisk for password display
* Update known issues

Signed-off-by: Mark Hilbush <[email protected]>
Signed-off-by: Maximilian Hess <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[squeezebox] Local cover art fails to download when auth enabled on LMS
4 participants